-
Notifications
You must be signed in to change notification settings - Fork 343
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: added notification doc for course author and open edx student #2172
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update all heading styles to match the current documentation standard: https://docs.openedx.org/en/latest/documentors/references/quick_reference_rst.html#headings
Could you please provide an example of what the desired format should be? I'm seeking assistance from the reference document titled "en_us/open_edx_students/source/SFD_notes.rst". instead of should i use this |
It looks like you fixed the thing I requested in the duplicate PR so I think this is fine to close. Out of curiousity, how is this page being used in the product tour? Are you linking from the notifications code to these published pages in some way? Should there be comments here so that we can understand the impact of removing or renaming these pages in the future? |
Its a duplicate PR. #2176 |
INF-1017
Description
I added section Managing Notifications in both course_authors and open_edx_students
The content of this section is presented in this doc: https://docs.google.com/document/d/1xNtDBQWqiKe0H_iukkN7hr77Evljs3Zn4Fc-ZXBUM-g/edit - Connect your Google account
Testing